Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ListMissingOneImplCounts spanner method #876

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

jcscottiii
Copy link
Collaborator

This is the spanner method that will power the missing one implementation graph.

It uses the pre calculated results from the BrowserFeatureSupportEvents table.

Features:

  • It allows the caller to pass in arbitrary set of browsers to compare against.
  • It has a string pagination token that is decoded and used to continue paging for results
  • It returns the results in DESC order of the Release Date

This method is similar to:

  1. the ListMetricsForFeatureIDBrowserAndChannel method used to get the results for the WPT metrics that power the chart on the feature detail, and
  2. the ListBrowserFeatureCountMetric method used to get the results for the feature count that power the chart on the stats page

This PR is a split up of https://github.com/GoogleChrome/webstatus.dev/compare/missing-one-impl-count-db?expand=1

Fixes #864

Depends on #872

@jcscottiii jcscottiii force-pushed the jcscottiii/store-events-methods branch from 087bdda to 352b19a Compare November 5, 2024 21:59
Base automatically changed from jcscottiii/store-events-methods to main November 5, 2024 22:42
This is the spanner method that will power the missing one implementation graph.

It uses the pre calculated results from the BrowserFeatureSupportEvents table.

Features:
- It allows the caller to pass in arbitrary set of browsers to compare against.
- It has a string pagination token that is decoded and used to continue paging for results
- It returns the results in DESC order of the Release Date

This method is similar to:
1. the [ListMetricsForFeatureIDBrowserAndChannel](https://github.com/GoogleChrome/webstatus.dev/blob/dd754efe5e2fa27cc80542fbacd41886e765b7ba/lib/gcpspanner/wpt_run_feature_metric.go#L418-L494)
    method used to get the results for the WPT metrics that power the chart
    on the feature detail, and
2. the [ListBrowserFeatureCountMetric](https://github.com/GoogleChrome/webstatus.dev/blob/dd754efe5e2fa27cc80542fbacd41886e765b7ba/lib/gcpspanner/browser_feature_count.go#L38C18-L105)
    method used to get the results for the feature count that power the chart
    on the stats page

This PR is a split up of https://github.com/GoogleChrome/webstatus.dev/compare/missing-one-impl-count-db?expand=1

Fixes #864

Depends on #872
@jcscottiii jcscottiii added this pull request to the merge queue Nov 8, 2024
Merged via the queue into main with commit 346acf3 Nov 8, 2024
7 checks passed
@jcscottiii jcscottiii deleted the jcscottiii/missing-one-impl branch November 8, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add spanner logic to query pages of missing one implementation count metrics
2 participants